Skip to content

Gtf creation #40

Merged
merged 21 commits into from
Jan 8, 2019
Merged

Gtf creation #40

merged 21 commits into from
Jan 8, 2019

Conversation

SebastianBeyvers
Copy link
Collaborator

Improved GTF-Creation Fixed a lot of bugs and formatting issues. More structured comments.

@SebastianBeyvers SebastianBeyvers added bug Something isn't working enhancement New feature or request labels Jan 6, 2019
@SebastianBeyvers SebastianBeyvers self-assigned this Jan 6, 2019
@SebastianBeyvers SebastianBeyvers removed their assignment Jan 6, 2019
Copy link
Collaborator

@renewiegandt renewiegandt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Minor changes of documentation.
going to add Anastasiia as a reviewer because she wrote the review of your scripts.

bin/3.1_create_gtf/Modules/Validator.py Show resolved Hide resolved
def __init__(self, out_file):

# Constructor
# input_parameter: out_file = Path to Outputfile
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

output file

bin/3.1_create_gtf/Modules/SaveResults.py Show resolved Hide resolved
# is_data_dir = is wd data_dir or not

# return_value: Link to Chainfile for conversion.
# Custom chainfiles and chainfiles for more organism can be specified in this section
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

chain files

# wd = working directory
# is_data_dir = is wd data_dir or not

# return_value: Link to Chainfile for conversion.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

chain file


def test_read_file(self):

# Method to testread the file
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

'testread' sounds and looks strange. Maybe "Method to test file format" would be a better comment.

# Find activity categories in bed file
# input_parameter: line = current line from bedfile
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

BED-file

# Method to get ucsc-celltype categories from JSON config
# input_parameter: organism = organism parameter
# wd = working directory, to find config fil
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

config file

# input_parameter: organism = input organism
# release = used Ensembl release
# wd = working directory (default is ".")
# data_dir = data directory (if specified this is used)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe you should write what is used if it is not specified

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I meant data_dir

Copy link
Collaborator

@renewiegandt renewiegandt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think you got your cursor at the wrong position at some point

# input_parameter: organism = input organism
# release = used Ensembl release
# wd = working directory (default is ".")
# data_dir = data directory (if specified this is used)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I meant data_dir

@SebastianBeyvers SebastianBeyvers merged commit e4d5c5c into dev Jan 8, 2019
Sign in to join this conversation on GitHub.
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants